Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.95%. Comparing base (f503097) to head (4074ee8).
✅ All tests successful. No failed tests found.

@@            Coverage Diff             @@
##             main    #6563      +/-   ##
==========================================
+ Coverage   89.92%   89.95%   +0.03%     
==========================================
  Files         381      381              
  Lines      107620   107718      +98     
  Branches   107620   107718      +98     
==========================================
+ Hits        96778    96900     +122     
+ Misses       7170     7151      -19     
+ Partials     3672     3667       -5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Merging this PR will not alter performance

✅ 50 untouched benchmarks

_{Comparing bma/fixCanBeRepliedTo (4074ee8) with main (f503097)}

If self.content.is_message() returns false, we don't need this change, right?

If self.content.is_message() returns false, we don't need this change, right?

No, because self.latest_json().is_some() is also returning true for LLS events.

is_message checks the type is MsgLike but also checks that the MsgLikeKind is Message, not LiveLocation, so that is why it returns false for live locations.

    /// Check whether this item's content is a
    /// [`Message`][MsgLikeKind::Message].
    pub fn is_message(&self) -> bool {
        matches!(self, Self::MsgLike(MsgLikeContent { kind: MsgLikeKind::Message(_), .. }))
    }

I think it is sensible to say that live location messages can't be replied to, and this looks like a reasonable way to do it.

Along with tests, I would like the comment to explain itself a little more. Maybe something like "Live location messages are not displayed on the timeline individually, so can't be replied to."

Thanks. I asked AI to add tests.